Skip to content

Add ArchUnitPython to Python unit tests#13367

Open
fniessink wants to merge 1 commit into
masterfrom
13241-add-archunit-to-checks
Open

Add ArchUnitPython to Python unit tests#13367
fniessink wants to merge 1 commit into
masterfrom
13241-add-archunit-to-checks

Conversation

@fniessink
Copy link
Copy Markdown
Member

Closes #13241.

@fniessink fniessink linked an issue Jun 2, 2026 that may be closed by this pull request
@fniessink fniessink force-pushed the 13241-add-archunit-to-checks branch 6 times, most recently from e3de77e to f03f248 Compare June 3, 2026 15:22
@@ -0,0 +1,18 @@
disable_logging # unused function (src/shared_test_code/__init__.py)
@@ -0,0 +1,18 @@
disable_logging # unused function (src/shared_test_code/__init__.py)
package_names # unused function (src/shared_test_code/__init__.py)
@@ -0,0 +1,18 @@
disable_logging # unused function (src/shared_test_code/__init__.py)
package_names # unused function (src/shared_test_code/__init__.py)
path_glob # unused function (src/shared_test_code/__init__.py)
disable_logging # unused function (src/shared_test_code/__init__.py)
package_names # unused function (src/shared_test_code/__init__.py)
path_glob # unused function (src/shared_test_code/__init__.py)
METRIC_ID # unused variable (src/shared_test_code/fixtures.py)
package_names # unused function (src/shared_test_code/__init__.py)
path_glob # unused function (src/shared_test_code/__init__.py)
METRIC_ID # unused variable (src/shared_test_code/fixtures.py)
METRIC_ID2 # unused variable (src/shared_test_code/fixtures.py)
path_glob # unused function (src/shared_test_code/__init__.py)
METRIC_ID # unused variable (src/shared_test_code/fixtures.py)
METRIC_ID2 # unused variable (src/shared_test_code/fixtures.py)
METRIC_ID3 # unused variable (src/shared_test_code/fixtures.py)
METRIC_ID # unused variable (src/shared_test_code/fixtures.py)
METRIC_ID2 # unused variable (src/shared_test_code/fixtures.py)
METRIC_ID3 # unused variable (src/shared_test_code/fixtures.py)
METRIC_ID4 # unused variable (src/shared_test_code/fixtures.py)
Comment thread tests/shared_test_code/.vulture-whitelist.py Fixed
Comment thread tests/shared_test_code/.vulture-whitelist.py Fixed
Comment thread tests/shared_test_code/.vulture-whitelist.py Fixed
@fniessink fniessink force-pushed the 13241-add-archunit-to-checks branch 2 times, most recently from 25558b0 to 3840d7e Compare June 3, 2026 19:22
Comment thread tests/shared_test_code/.vulture-whitelist.py Fixed
Comment thread tests/shared_test_code/.vulture-whitelist.py Fixed
Comment thread tests/shared_test_code/.vulture-whitelist.py Fixed
@fniessink fniessink force-pushed the 13241-add-archunit-to-checks branch 2 times, most recently from b094eab to f053755 Compare June 3, 2026 21:05
METRIC_ID2 # unused variable (src/shared_test_code/fixtures.py:8)
METRIC_ID3 # unused variable (src/shared_test_code/fixtures.py:9)
METRIC_ID4 # unused variable (src/shared_test_code/fixtures.py:10)
REPORT_ID # unused variable (src/shared_test_code/fixtures.py:11)
METRIC_ID3 # unused variable (src/shared_test_code/fixtures.py:9)
METRIC_ID4 # unused variable (src/shared_test_code/fixtures.py:10)
REPORT_ID # unused variable (src/shared_test_code/fixtures.py:11)
REPORT_ID2 # unused variable (src/shared_test_code/fixtures.py:12)
METRIC_ID4 # unused variable (src/shared_test_code/fixtures.py:10)
REPORT_ID # unused variable (src/shared_test_code/fixtures.py:11)
REPORT_ID2 # unused variable (src/shared_test_code/fixtures.py:12)
SOURCE_ID # unused variable (src/shared_test_code/fixtures.py:13)
@fniessink fniessink force-pushed the 13241-add-archunit-to-checks branch 2 times, most recently from 150972d to 2e62b6a Compare June 4, 2026 08:12
- Adopted ArchUnitPython and added foundation-layer architecture tests (utilities package depends on nothing internal, no cycles, etc.)
- Created a shared test-helper package (`tests/shared_test_code/`) to house shared test code such as `load_data_model`, `disable_logging`, and the METRIC_ID/REPORT_ID/SOURCE_ID/SUBJECT_ID constants
- Wired the package into components as a dev dependency and added it to CI/CD workflows

Closes #13241.
@fniessink fniessink force-pushed the 13241-add-archunit-to-checks branch from 2e62b6a to 7abff6b Compare June 4, 2026 08:23
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Jun 4, 2026

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add ArchUnit to checks

1 participant